Skip to content

Add streaming parameter files - #910

Open
LeonDong0807 wants to merge 1 commit into
sharkdp:masterfrom
LeonDong0807:agent/parameter-file-streaming
Open

Add streaming parameter files#910
LeonDong0807 wants to merge 1 commit into
sharkdp:masterfrom
LeonDong0807:agent/parameter-file-streaming

Conversation

@LeonDong0807

Copy link
Copy Markdown

Closes #813

Summary

  • add --parameter-file VAR FILE for reading parameter values from file lines
  • process values with buffered, line-by-line I/O instead of materializing the file contents
  • support both LF and CRLF line endings
  • preserve command naming, reference-command ordering, and per-command export behavior
  • document the new option and add unit/integration coverage, including a large input case

Validation

  • cargo fmt --all -- --check
  • cargo clippy --locked --all-targets (only pre-existing warnings)
  • cargo test --locked — 105 tests passed

@LeonDong0807
LeonDong0807 marked this pull request as ready for review July 16, 2026 13:35

Copy link
Copy Markdown
Author

The current CI run is marked action_required, which appears to be waiting for approval because this is my first contribution to the repository. When convenient, could a maintainer approve the workflow run? The full local suite passes (105 tests), and I’ll address any CI or review feedback promptly.

@Scripter17

Copy link
Copy Markdown

I should've checked this before setting the bounty, but the last 3 commits to Hyperfine were 3, 5, and 8 months ago. It seems to be largely abandoned and/or done. There's a decent chance Sharkdp will just never get to this.

So how about this: I'll check out this patch at some point soon, and if I find it good and Sharkdp doesn't respond by, idk August 18th, I'll just pay you the bounty anyway. Technically the agreement was payment upon merge but it would be just rude for me to not pay out such a small bounty over something you can't control.

@LeonDong0807

LeonDong0807 commented Jul 18, 2026 via email

Copy link
Copy Markdown
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow getting parameters from file lines

2 participants